Skip to content

feat: Flush after reading the archive from disk#4862

Closed
another-rex wants to merge 2 commits into
google:masterfrom
another-rex:read-and-flush
Closed

feat: Flush after reading the archive from disk#4862
another-rex wants to merge 2 commits into
google:masterfrom
another-rex:read-and-flush

Conversation

@another-rex

Copy link
Copy Markdown
Contributor

Attempt at advising the kernel that we don't need that file in cache anymore

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Comment thread go/cmd/gitter/gitter.go Fixed
Comment thread go/cmd/gitter/gitter.go
fileData := stdout.Bytes()

// Write to disk
f, err := os.OpenFile(archivePath, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0644)

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a
user-provided value
.
Comment thread go/cmd/gitter/gitter.go
}

fileData, err := os.ReadFile(archivePath)
f, err := os.Open(archivePath)

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a
user-provided value
.
@github-actions

Copy link
Copy Markdown

This pull request has not had any activity for 60 days and will be automatically closed in two weeks

@github-actions github-actions Bot added the stale The issue or PR is stale and pending automated closure label Apr 21, 2026
@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown

Automatically closing stale pull request

@github-actions github-actions Bot added the autoclosed Closed by automation label May 5, 2026
@github-actions github-actions Bot closed this May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autoclosed Closed by automation stale The issue or PR is stale and pending automated closure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants